Skip to content

Add blog post author bylines#280

Merged
adamltyson merged 3 commits into
mainfrom
codex/blog-post-bylines
May 27, 2026
Merged

Add blog post author bylines#280
adamltyson merged 3 commits into
mainfrom
codex/blog-post-bylines

Conversation

@PolarBean

@PolarBean PolarBean commented May 19, 2026

Copy link
Copy Markdown
Contributor

This is a copy of the changes which were reviewed originally here (the changes were copied over using codex)
brainglobe/brainglobe.github.io#496
I have built it locally and it is functionally the same.
image

@PolarBean PolarBean changed the title [codex] Add blog post author bylines Add blog post author bylines May 19, 2026
@PolarBean PolarBean marked this pull request as ready for review May 19, 2026 18:36
@niksirbi

Copy link
Copy Markdown
Member

This is nice! But I just found out there is an alternative way of doing this, see comment. Opinions welcome!

@adamltyson

Copy link
Copy Markdown
Member

As discussed elsewhere, although this custom approach is nicer, we should go for the sphinx-native approach for maintainability. Once neuroinformatics-unit/movement#990 is merged, it can be copied here.

@niksirbi

Copy link
Copy Markdown
Member

I reverted the custom author byline commit and replaced it with ABlog's built-in postcard sidebar templates, following the same approach used in neuroinformatics-unit/movement#990. See discussion under the movement PR for context on how we reached this decision.

Together with brainglobe/brainglobe.github.io#503, this makes all our blogs have the same sidebar settings.

Preview for blog/index:

image

Preview for blogpost page:

image

Changes

  • Remove the add_blog_author_byline Sphinx extension hook and its imports from conf.py
  • Remove the .blog-post-author CSS block from custom.css
  • Add html_sidebars config to use ABlog's native postcard.html + recentposts.html templates on blog post pages, and authors.html + archives.html on the blog index
  • Set fontawesome_included = True so ABlog renders icons via the FontAwesome already bundled by the PyData theme

@niksirbi niksirbi requested a review from adamltyson May 24, 2026 10:35
@adamltyson adamltyson merged commit 5c58442 into main May 27, 2026
4 checks passed
@adamltyson adamltyson deleted the codex/blog-post-bylines branch May 27, 2026 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants